Skip to content
This repository has been archived by the owner on May 1, 2023. It is now read-only.

Fixed intermittent errors at fetch time #24

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pblanco-dekalabs
Copy link

We've detected intermittent errors through our tests, when consuming the testnet.

Seems that somehow the indexer gets overflown by too many parallel requests, and throws 503 errors at random.
To solve this, an await was added inside the loop.

@barnjamin
Copy link
Owner

Thanks! This would mean getting the listings is slower, since each would have to wait for the last to finish before it gets fetched.

Maybe some secondary loop that packs up promises 5 or 10 at a time depending?

Also, I've not had time recently to get to the frond end changes for this yet but the backend contracts are working for https://github.com/barnjamin/rareaf/tree/app-addr. They're vastly simplified but require Teal 6 (just merged to go-algorand/master but not on mainnet yet) since we need rekey inside the application.

@pblanco-dekalabs
Copy link
Author

That's a good idea! I'll look to pack requests to avoid slowness.

@barnjamin
Copy link
Owner

There is another branch that I've been working on to rework the logic, i'd wait until that is done to even consider going to mainnet

@barnjamin
Copy link
Owner

https://github.com/barnjamin/rareaf/tree/app-addr havent had time to finish it up

@barnjamin
Copy link
Owner

hey @scottbolasevich still haven't found time to work on this 😬 but feel free to pop into the Algorand Discord with any questions, im barnji on there.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants